Skip to content

#17 add icons to card#27

Open
nattydodd wants to merge 3 commits intodillionverma:masterfrom
nattydodd:#17_add-icons-to-card
Open

#17 add icons to card#27
nattydodd wants to merge 3 commits intodillionverma:masterfrom
nattydodd:#17_add-icons-to-card

Conversation

@nattydodd
Copy link
Copy Markdown

screen shot 2018-10-12 at 9 21 47 pm

@dillionverma
Copy link
Copy Markdown
Owner

Do you think you can make a separate component in /src/features/issues/components/Icon.js and maybe abstract that part of the code a bit.

@gregorysl had a good example of that here.

Also, since you already came so far with this, can you also add the icon for PR and merged PR with green and purple respectively. If not then no problem, I can open another issue 😄

@gregorysl
Copy link
Copy Markdown
Contributor

Wont it'll be faster to merge mine, and apply @nattydodd style changes?

@dillionverma
Copy link
Copy Markdown
Owner

Yes it would @gregorysl. Lets merge yours in first then apply styles from this one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants